-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add unprivileged upgrade testing for Windows. #4642
Add unprivileged upgrade testing for Windows. #4642
Conversation
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
…d-upgrade-testing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM besides a couple of nits.
|
Force merging as SonarQube is reporting 0% but new code added in the PR is not unit testable and integration tests have been updated. |
* Add unprivileged upgrade testing for Windows. * Adjust to 8.14.0-SNAPSHOT as minimum. * Add SeCreateSymbolicLinkPrivilege * Simplify if unprivileged mode is supported. * Fix file access for upgrade. * Fix kill watcher process on Windows in unprivileged mode. * Fix imports. * Fix kill non child process in tests. * Fix lint. (cherry picked from commit 4d3a3f9)
* Add unprivileged upgrade testing for Windows. * Adjust to 8.14.0-SNAPSHOT as minimum. * Add SeCreateSymbolicLinkPrivilege * Simplify if unprivileged mode is supported. * Fix file access for upgrade. * Fix kill watcher process on Windows in unprivileged mode. * Fix imports. * Fix kill non child process in tests. * Fix lint. (cherry picked from commit 4d3a3f9) Co-authored-by: Blake Rouse <blake.rouse@elastic.co>
What does this PR do?
Adds upgrade testing for unprivileged mode on Windows.
Why is it important?
Upgrade testing for unprivileged mode on Windows was not enabled.
Checklist
[ ] I have made corresponding changes to the documentation[ ] I have made corresponding change to the default configuration files[ ] I have added an entry in./changelog/fragments
using the changelog toolRelated issues